Skip to content

Conversation

grdsdev
Copy link
Contributor

@grdsdev grdsdev commented Oct 7, 2025

Summary

Fixes Swift Package Manager warnings about unhandled snapshot files by explicitly excluding __Snapshots__ directories from test targets.

Changes

  • Added exclude: ["__Snapshots__"] to the following test targets in Package.swift:
    • FunctionsTests (5 snapshot files)
    • PostgRESTTests (32 snapshot files)
    • StorageTests (1 snapshot file)

Why

These snapshot files are generated and managed by swift-snapshot-testing and should not be processed as test resources. Without explicit exclusion, Swift Package Manager issues warnings about unhandled files.

Test plan

  • swift build completes without warnings about unhandled files
  • Tests still run successfully with snapshot testing functionality intact

🤖 Generated with Claude Code

Explicitly exclude __Snapshots__ directories from FunctionsTests,
PostgRESTTests, and StorageTests targets to resolve Swift Package
Manager warnings about unhandled files.

These snapshot files are managed by swift-snapshot-testing and should
not be processed as test resources.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
@grdsdev grdsdev requested a review from a team October 7, 2025 19:00
@grdsdev grdsdev enabled auto-merge (squash) October 7, 2025 19:00
@grdsdev grdsdev changed the title fix: exclude __Snapshots__ directories from test targets test: exclude __Snapshots__ directories from test targets Oct 7, 2025
@coveralls
Copy link

Pull Request Test Coverage Report for Build 18323189729

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 77.292%

Totals Coverage Status
Change from base Build 18320538177: 0.0%
Covered Lines: 5875
Relevant Lines: 7601

💛 - Coveralls

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants